-
Notifications
You must be signed in to change notification settings - Fork 14.1k
Run clippy on cg_gcc in CI #149458
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Run clippy on cg_gcc in CI #149458
Conversation
|
|
| --set llvm.libzstd=true \ | ||
| --set 'rust.codegen-backends=[\"llvm\",\"gcc\"]' | ||
| ENV SCRIPT python3 ../x.py \ | ||
| ENV SCRIPT python3 ../x.py --stage 2 clippy compiler/rustc_codegen_gcc && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this test both with and without --no-default-features?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No it doesn't. I can add that as a follow-up in the clippy rule.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice since the above issue would only happens without the master feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
8578319 to
43bd395
Compare
This comment has been minimized.
This comment has been minimized.
43bd395 to
0626c7a
Compare
|
Fixed bootstrap tests and CI is happy. :) |
This is to prevent this issue: in cg_gcc repository, we run clippy on our code but not in here, which can create issues.
cc @antoyo
r? @Kobzol